Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade OVS to 2.17.7 #5225

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Conversation

antoninbas
Copy link
Contributor

This is the latest patch version available for OVS 2.17.

We also fix a bug in the Dockerfile which installs OVS using .deb packages built from source. The command that was used would try to upgrade the openvswitch-* packages if a more recent version was available in upstream repos, which would cause the installation to fail. For now, we prefer to use the packages that we build from source, as we have more granular control over which version gets installed. This issue is fixed by using apt instead of dpkg to install the .deb packages (this is supported in "recent" apt versions).

Thanks @Dyanngg for reporting the issue.

This is the latest patch version available for OVS 2.17.

We also fix a bug in the Dockerfile which installs OVS using .deb
packages built from source. The command that was used would try to
upgrade the openvswitch-* packages if a more recent version was
available in upstream repos, which would cause the installation to
fail. For now, we prefer to use the packages that we build from source,
as we have more granular control over which version gets installed. This
issue is fixed by using apt instead of dpkg to install the .deb packages
(this is supported in "recent" apt versions).

Thanks @Dyanngg for reporting the issue.

Signed-off-by: Antonin Bas <[email protected]>
@antoninbas antoninbas requested review from tnqn and Dyanngg July 11, 2023 00:44
@luolanzone luolanzone added the action/release-note Indicates a PR that should be included in release notes. label Jul 11, 2023
Copy link
Member

@tnqn tnqn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antoninbas
Copy link
Contributor Author

/test-all

@xliuxu xliuxu added the action/backport Indicates a PR that requires backports. label Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/backport Indicates a PR that requires backports. action/release-note Indicates a PR that should be included in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants